Skip to content

chore: updated 0.5 docs, added missing getGeneratedTokenCount to LLMModule#632

Merged
mkopcins merged 1 commit intomainfrom
@mkopcins/tokens-batching-fix
Oct 2, 2025
Merged

chore: updated 0.5 docs, added missing getGeneratedTokenCount to LLMModule#632
mkopcins merged 1 commit intomainfrom
@mkopcins/tokens-batching-fix

Conversation

@mkopcins
Copy link
Copy Markdown
Collaborator

@mkopcins mkopcins commented Oct 2, 2025

Description

Updated 0.5 versioned docs with token batching
Added getGeneratedTokenCount to LLMModule.ts

Introduces a breaking change?

  • Yes
  • No

Type of change

  • Bug fix (change which fixes an issue)
  • New feature (change which adds functionality)
  • Documentation update (improves or adds clarity to existing documentation)
  • Other (chores, tests, code style improvements etc.)

Tested on

  • iOS
  • Android

Testing instructions

Screenshots

Related issues

Checklist

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated the documentation accordingly
  • My changes generate no new warnings

Additional notes

@mkopcins mkopcins marked this pull request as ready for review October 2, 2025 08:04
@mkopcins mkopcins merged commit 059b3b8 into main Oct 2, 2025
5 checks passed
@mkopcins mkopcins deleted the @mkopcins/tokens-batching-fix branch October 2, 2025 08:17
mkopcins added a commit that referenced this pull request Oct 2, 2025
…odule (#632)

## Description

Updated 0.5 versioned docs with token batching
Added getGeneratedTokenCount to LLMModule.ts

### Introduces a breaking change?

- [ ] Yes
- [x] No

### Type of change

- [x] Bug fix (change which fixes an issue)
- [ ] New feature (change which adds functionality)
- [x] Documentation update (improves or adds clarity to existing
documentation)
- [ ] Other (chores, tests, code style improvements etc.)

### Tested on

- [x] iOS
- [x] Android

### Testing instructions

<!-- Provide step-by-step instructions on how to test your changes.
Include setup details if necessary. -->

### Screenshots

<!-- Add screenshots here, if applicable -->

### Related issues

<!-- Link related issues here using #issue-number -->

### Checklist

- [ ] I have performed a self-review of my code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have updated the documentation accordingly
- [ ] My changes generate no new warnings

### Additional notes

<!-- Include any additional information, assumptions, or context that
reviewers might need to understand this PR. -->

Co-authored-by: Mateusz Kopciński <mateusz.kopcinski@swmansnion.com>
KnextKoder pushed a commit to Synkhiv/react-native-executorch that referenced this pull request Nov 7, 2025
…odule (software-mansion#632)

## Description

Updated 0.5 versioned docs with token batching
Added getGeneratedTokenCount to LLMModule.ts

### Introduces a breaking change?

- [ ] Yes
- [x] No

### Type of change

- [x] Bug fix (change which fixes an issue)
- [ ] New feature (change which adds functionality)
- [x] Documentation update (improves or adds clarity to existing
documentation)
- [ ] Other (chores, tests, code style improvements etc.)

### Tested on

- [x] iOS
- [x] Android

### Testing instructions

<!-- Provide step-by-step instructions on how to test your changes.
Include setup details if necessary. -->

### Screenshots

<!-- Add screenshots here, if applicable -->

### Related issues

<!-- Link related issues here using #issue-number -->

### Checklist

- [ ] I have performed a self-review of my code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have updated the documentation accordingly
- [ ] My changes generate no new warnings

### Additional notes

<!-- Include any additional information, assumptions, or context that
reviewers might need to understand this PR. -->

Co-authored-by: Mateusz Kopciński <mateusz.kopcinski@swmansnion.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants